Skip to content

Conversation

zingo
Copy link
Collaborator

@zingo zingo commented Apr 4, 2025

This fix a bug that when you used non default memory mode the PTE was correctly build but the elf was still build with the default mode.

Change-Id: I1a7a70d6a161c740a2562814acc2cc7e99b410f5

cc @digantdesai @freddan80 @per @oscarandersson8218

This fix a bug that when you used non default memory mode the PTE was
correctly build but the elf was still build with the default mode.

Signed-off-by: Zingo Andersen <[email protected]>
Change-Id: I1a7a70d6a161c740a2562814acc2cc7e99b410f5
@zingo zingo requested a review from digantdesai as a code owner April 4, 2025 10:07
Copy link

pytorch-bot bot commented Apr 4, 2025

🔗 Helpful Links

🧪 See artifacts and rendered test results at hud.pytorch.org/pr/pytorch/executorch/9902

Note: Links to docs will display an error until the docs builds have been completed.

✅ You can merge normally! (1 Unrelated Failure)

As of commit d25de8a with merge base 4717459 (image):

UNSTABLE - The following job is marked as unstable, possibly due to flakiness on trunk:

This comment was automatically generated by Dr. CI and updates every 15 minutes.

@facebook-github-bot facebook-github-bot added the CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. label Apr 4, 2025
@zingo zingo added partner: arm For backend delegation, kernels, demo, etc. from the 3rd-party partner, Arm ciflow/trunk topic: not user facing labels Apr 4, 2025
@zingo
Copy link
Collaborator Author

zingo commented Apr 4, 2025

qnn fail is unrelated

@zingo zingo merged commit 2b4e525 into pytorch:main Apr 4, 2025
166 of 169 checks passed
kirklandsign pushed a commit that referenced this pull request Apr 11, 2025
This fix a bug that when you used non default memory mode the PTE was
correctly build but the elf was still build with the default mode.

Signed-off-by: Zingo Andersen <[email protected]>
@zingo zingo deleted the Arm-backend-Handle-memory-mode-corectly-in-test_model.py branch May 16, 2025 19:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ciflow/trunk CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. partner: arm For backend delegation, kernels, demo, etc. from the 3rd-party partner, Arm topic: not user facing
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants